Merge branch 'text-undo-test' into 'master'

Add another text history test

See merge request GNOME/gtk!3713

Add another text history test

Add a tests that checks we group undo actions as expected
when the user is typing.

Merge branch 'wip/chergert/fix-texthistory-grouping' into 'master'

texthistory: hoist single actions from group

Closes gnome-text-editor#97

See merge request GNOME/gtk!3711

texthistory: hoist single actions from group

In many cases across GtkTextBuffer, we end up with operations performed
inside of a begin/end user action. Those can be coalesced into a single
sub-action within the group, and hoisted out of the group. Doing so
increases the chances that we chain similar actions together for words.

Additionally, this fixes an issue introduced in
6179886b14eccd3034dc8121aac7b709aac073ec for #3977 where GNOME Text Editor
started to group all possible actions into a single group.

Fixes GNOME/gnome-text-editor#97

icontheme: Fix binding annotation

Merge branch 'wayland-cursor-theme' into 'master'

wayland: Look for cursor themes in $HOME

Closes #4080

See merge request GNOME/gtk!3708

wayland: Look for cursor themes in $HOME

We should look in the same places that libXcursor does,
so add $XDG_DATA_HOME/icons and $HOME/.icons to the list.

Fixes: #4080

Merge branch 'more-vbo-overflow' into 'master'

gsk: Use the right limit for batch size

See merge request GNOME/gtk!3707

gsk: Another vbo_size overflow fix

We can overflow vbo_size not just by batching
too much, but also by producing humongous text
nodes. Split them up.

gsk: Use the right limit for batch size

We use 16 bits, so G_MAXINT16 is one bit short.
Just make it explicit as 0xffff.

Merge branch 'hello-world-fixup' into 'master'

Update hello-world.c

See merge request GNOME/gtk!3706

Update hello-world.c

Simplify the code a bit and make the window
match the existing screenshot.

Merge branch 'avoid-batch-size-overflow' into 'master'

Revert "gsk: Respect max element vertices limitation"

See merge request GNOME/gtk!3704

Merge branch 'wip/exalm/buttons' into 'master'

GtkMenuButton icon+arrow support + fixes

Closes #3501

See merge request GNOME/gtk!3694

Merge branch 'gtklistitemfactory-autocleanup' into 'master'

gtk: Define the GtkListItemFactory autocleanup func

See merge request GNOME/gtk!3703

gsk: Don't overflow vbo_count

We use 16 bits to store vbo_count, so we can't create
batches that have more than 65535 vertices. Pay attention
to that limit when merging batches.

Revert "gsk: Respect max element vertices limitation"

This reverts commit f58fc6b22e143e43b45f297795959f217a26d95f.

gtk: Define the GtkListItemFactory autocleanup func

Merge branch 'ebassi/issues-4076' into 'master'

docs: Use code blocks for inline XML

Closes #4076

See merge request GNOME/gtk!3702

docs: Use code blocks for inline XML

Otherwise the markdown renderer will just ignore them.

Fixes: #4076

Update Portuguese translation

(cherry picked from commit a9c68757b5c339ea4a17a03d4f20edef197c717b)

menubutton: Don't hardcode label<->arrow spacing

This should come from the theme instead, as it does for icon<->arrow.

menubutton: Add a way to always show arrow with image buttons

Make sure the button still has the .image-button style class with an icon,
also add it to the initial state with only an arrow. Add a new
.arrow-button style class for the icon+arrow state so it's possible to
style it.

Remove spacing from the label+arrow variant to match, re-add it from the
stylesheet for both.

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/3501

menubutton: Remove a random g_return_if_fail()

There's already one above.

menubutton: Reset label when setting icon name and vice versa

Match GtkButton.

button: Reset style classes when setting arbitrary children

Don't leave leftover .image-button or .text-button.

Merge branch 'wip/sadiq/reveal-controls-on-touch' into 'master'

video: Reveal controls on tap

See merge request GNOME/gtk!3648

Merge branch 'wip/chergert/gdk-macos-gdkdrop' into 'master'

GdkDrop suport for macOS backend

See merge request GNOME/gtk!3683

Merge branch 'dnd-fixes' into 'master'

dnd: Add another assertion

See merge request GNOME/gtk!3689

Merge branch 'ebassi/filechooser-no-trash' into 'master'

Never show the Trash in the FileChooserWidget side bar

Closes #674

See merge request GNOME/gtk!3692

Merge branch 'checkbutton-release-outside' into 'master'

checkbutton: Match GtkButton behavior

Closes #4061

See merge request GNOME/gtk!3691

Merge branch 'fix_tests_testdnd' into 'master'

make testdnd trashcan work again

See merge request GNOME/gtk!3700

Merge branch 'macos-input-method' into 'master'

macos: fix the position of curosr of input method for macos.

Closes #4063

See merge request GNOME/gtk!3699

make testdnd trashcan work again

so that dragging into the 'trashcan' is detected

Merge branch 'gtklistitem-autocleanup' into 'master'

gtk: Define the GtkListItem autocleanup func

See merge request GNOME/gtk!3698

Merge branch 'migration_patch' into 'master'

Add section about `gtk_widget_set_app_paintable`

Closes #4007

See merge request GNOME/gtk!3641

Add section about `gtk_widget_set_app_paintable`

macos: fix the position of curosr of input method for macos.

In the refactoring from GdkWindow to GdkSurface, GtkWidget no longer
corresponds to a GdkSurface. We have to calculate the relative position
from GtkWidget to the GdkSurface.

Closes #4063.

gtk: Define the GtkListItem autocleanup func

Merge branch 'fix-pointer-gestures-kde-plasma' into 'master'

gdk/wayland: Fix pointer-gestures version selection

See merge request GNOME/gtk!3693

gdk/wayland: Fix pointer-gestures version selection

version == GDK_ZWP_POINTER_GESTURES_V1_VERSION will fail if the
compositor implements version 2 of pointer-gestures-v1.

Never show the Trash in the FileChooserWidget side bar

The Trash is a special location: files cannot be copied or moved, there,
and the file selection dialog is not able to restore files from the
Trash.

Fixes: #674

Revert "Never show the Trash in the FileChooserWidget side bar"

This reverts commit cd39e417e1f816e7c509f9c14e4a0b61fce81430.

Never show the Trash in the FileChooserWidget side bar

The Trash is a special location: files cannot be copied or moved, there,
and the file selection dialog is not able to restore files from the
Trash.

Fixes: #674

docs: Fix a typo

docs: Add a section about Drag-and-Drop

Merge branch 'wip/chergert/textview-access-to-contexts' into 'master'

textview: give application developers access to RTL and LTR context

See merge request GNOME/gtk!3690

checkbutton: Match GtkButton behavior

When leaving the widget before releasing the button,
we should not activate the checkbutton, to match the
way GtkButton behaves.

Fixes: #4061

textview: give application developers access to RTL and LTR context

This allows developers to modify the pango context that is used when
rendering text within the text view.

Such access can be useful to alter how rounding occurs with API such as
pango_context_set_round_glyph_positions() and is needed by GtkSourceView
for proper placement of glyphs within the overview map.

docs: Tweak wording

Instead of drag'n'drop, say drag-and-drop. That is easier
to read, and less unclear on the markdown syntax implications.

wayland: Fix some dnd corner case

We must call gdk_drag_drop_done() when the drag ends,
successfully or not. Without this, we get an unwarranted
emission of ::cancel after a successful drop.

Since only the first call to gdk_drag_drop_done() is taking
effect, it is safe to call as a fallback, after emitting
::dnd-finished. If the application connects to that signal
and calls gdk_drag_drop_done() itself, its call will take
precedence.

This matches what the X11 implementation does.

dnd: Add another assertion

Assert that gdk_drop_finish() is called after the
drop is performed.

Merge branch 'fix-3798' into 'master'

GDK-Win32: Fix drag surface positioning (issue #3798)

Closes #3798

See merge request GNOME/gtk!3659

Merge branch 'gsk-big-batches' into 'master'

ci: Build Broadway on MacOS

See merge request GNOME/gtk!3686

Merge branch 'matthiasc/for-master' into 'master'

ci: Build Broadway on MacOS

See merge request GNOME/gtk!3685

gsk: Respect max element vertices limitation

We are pretty good at batching commands now, and we can easily
produce batches that exceed the maximum number of elements per
draw call that the hw can handle. Query that number, and respect
it when merging batches.

This fixes the rendering of the overview map in GtkSourceView.

ci: Build Broadway on MacOS

We've seen a build failure if Broadway is built without
the X11 backend. Lets recreate that configuration in ci
so it doesn't break again.

gdkdrag-win32.c: Fix drag surface positioning

Determine the root_x and root_y coordinates of the drag surface by
relying on the coordinates of the surface where the drag is being
carried out, plus the coordinates that we receive from the drag event,
which is in-line with what the X11 backend does.

This will prevent the drag surface from being initially drawn at the
correct position, but jumping towards the top-left corner of the screen
shortly afterwards.

The DnD support will still need some more updates to function correctly
on Windows, but at least this is a small improvement.

Fixes issue #3798.

Merge branch 'matthiasc/for-master' into 'master'

treeviewcolumn: Clip header buttons

Closes #4045

See merge request GNOME/gtk!3682

macos: implement GdkDrop for macOS

This gets the basic mechanics of the drop portion of DnD working on the
macOS backend. You can drag, for example, from TextEdit into GNOME
Text Editor when using the macOS backend.

Other content formats are supported, and match what is currently
supported by the clipboard backend as the implementation to read
from the pasteboard is shared.

Currently, we look up the GdkDrag for the new GdkDrop. However,
nothing is stashing the drag away for further lookup. More work is
needed on GdkMacosDrag for that to be doable.

macos: make pasteboard usage reusable

We will want to be able to reuse the pasteboard reading code from
the macOS DnD drop backend. This just removes the pasteboard
bits from the implementation and allows that to be passed in as in
both clipboard and DnD cases we'll have a specific NSPasteboard
to read from.

macos: check for GdkMacosBaseView before using as such

We can get events for external windows such as those from native
file chooser dialogs.

treeviewcolumn: Clip header buttons

We don't want to overdraw when dragging a narrow column
around, and we also need the clipping to avoid picking
the wrong column, when a later column button overlaps
an earlier one.

Fixes: #4045

macos: Fix a compiler warning

macos: move drag motion to GdkMacosDrag

This doesn't help to be pushed off into the surface, as it makes things
more different than the X11 implementation.

macos: fix position of drag surface

macos: register known clipboard types for drag destination

Merge branch 'wip/chergert/fix-broadway-build' into 'master'

inspector: fix compilation with broadway

See merge request GNOME/gtk!3681

inspector: fix compilation with broadway

Merge branch 'matthiasc/for-master' into 'master'

action muxer fixes

See merge request GNOME/gtk!3678

Update Portuguese translation

(cherry picked from commit 807537dd5dc306edfae5abd82fbbb3e346f6d682)

testsuite: Avoid a compiler warning

This was showing up in ci on macos.

widget-factory: Show Ctrl-? in the menu

We use our own action for showing the shortcuts window,
so we need to associate this shortcut with our action
for it to show up.

shortcutcontroller: Don't reinject menu accels

The normal way to associate accels with actions is
to attach a shortcut controller to the widget. The shorcut
controller will inject the accel into the action muxer
tree, so that it can get displayed in widgets that activate
the action (say, in menus.

This approach does not works for generated menus, since the
widgets are not in the hands of the app developer, so attaching
shortcut controllers to them is impractical.

Instead, GtkModelButton has an accel property that gets
bound to the accel coming from the action muxer tree (most
likely put there via gtk_application_set_accel_for_action),
and creates a shortcut controller itself.

The change in this commit is to prevent the shortcut controller
from injecting the accel into the action muxer tree in this case.
Otherwise, the accels get 'stuck' and we won't update them if the
global accels are later changed.

This is a hack, and needs a better solution.

window: Propagate accel changes

When the global accels change, tell the
shortcut controller to re-inject them into
the action muxer hierarchy.

shortcutcontroller: Add a way to update accels

This will be used to make accel changes propagate.

action muxer: Propagate registrations further up

We were pretty aggressive about not registering
observers further than necessary, stopping at the
first muxer that provides an action.

But even though action changes further up in the tree
won't be relevant in that case, we need to listen to
accel changes, since they may come from higher up
in the tree (e.g. when using
gtk_application_set_accels_for_action with an action
that is defined on a widget.

So, register all the way to the top, and stop propagating
action changes when we hit a muxer that provides the action.

menutrackeritem: Be more careful with accel changes

In some cases (such as when getting a new parent), the
action muxer doesn't know exactly which detailed actions
have changed accels, so we call primary_accel_changed with
just an action name.

Make the menu tracker item handle that case by matching
either against the detailed name or the the action name.

action muxer: Propagate accel changes

When we set a new parent on an action muxer,
accels may change, so call primary_accel_changed
for involved actions.

Merge branch 'wip/otte/for-master' into 'master'

spinner: Behave like any other icon

See merge request GNOME/gtk!3677

spinner: Behave like any other icon

Take the size from -gtk-icon-size.

Note that min-width/height still works, as those properties are handled
by the generic widget sizing machinery in GTK4.

actionobserver: Clarify primary_accel_changed docs

Either action_name or action_and_target may be NULL
here, and observer implementations should be careful
when checking those values.

print-editor: Make accels work

Sadly, they don't show up anymore with this.

bloatpad: Add icons as resources

This can serve as a useful example for how to add
themed icons as resources.

bloatpad: Give the toolbar a facelift

Make it look more like a toolbar.

bloatpad: Improve accel dialog

Make this dialog a bit less of an eyesore.

examples: Use existing icons

The sunny icon doesn't exist anymore.

examples: Drop a redundant frame

The frame in the drawing example adds nothing
and looks wrong.

Merge branch 'fix-3793' into 'master'

gdksurface-win32.c: Fix up popup placement (fix issue #3793)

Closes #3793

See merge request GNOME/gtk!3674

Merge branch 'media-stream-name-collision' into 'master'

mediastream: Some more api renaming

Closes #4023

See merge request GNOME/gtk!3673

gdksurface-win32.c: Fix up popup placement

If we are undergoing a surface move, just apply the next_layout anyways,
even if we are not moving a toplevel surface.

Update the way how we obtain the x and y coordinates of a surface, if it
is a toplevel, apply the x and y coordinates from the results from we
obtained the underlying Win32 HWND, as we did before.  But if it is a
popup, use gdk_win32_surface_get_geometry() to obtain the correct x and
y coordinates to place our popup surface.

Also correct how we compute the shadow dimensions, and the final popup
rectangle as we attempt to layout the popup surface, since GDK-Win32
keeps track of the shadow dimensions in system (unscaled) units, not GDK
units.

Fixes issue #3793.

gdksurface-win32.c: Fix formatting for popup functions

Merge branch 'drop-target-name-collision' into 'master'

droptarget: Rename :drop to :current-drop

Closes #4028

See merge request GNOME/gtk!3669

mediastream: Some more api renaming

Also rename gtk_media_stream_ended to
gtk_media_stream_set_ended, to avoid naming
collision with GtkMediaStream:ended.

The existing entry points still exist, deprecated
and marked as non-introspectable.

Update all internal uses.

Fixes: #4023

droptarget: Rename :drop to :current-drop

Rename the GtkDropTraget:drop property to :current-drop,
to avoid naming collision with the signal of the same
name.

We leave the old property and getter in place, deprecated
and marked as non-introspectable.

Fixes: #4028

Merge branch 'cherry-pick-4ba89f25' into 'master'

cellarea: Don't shrink area too much

See merge request GNOME/gtk!3672

Merge branch 'label-underline-markup' into 'master'

label: Fix mnemonic handling

Closes #4041

See merge request GNOME/gtk!3670

cellarea: Don't shrink area too much

Do not compute rectangles with negative width/height. This avoids
assertion failures further down when those rectangles were actually
checked.

https://bugzilla.redhat.com/show_bug.cgi?id=1962215

(cherry picked from commit 4ba89f25b8a88616afc1915bdb4fb87d13efae6f)

Add a mnemonic testcase

Add a testcase that tests the label mnemonic handling
that was fixed in the previous commit.

label: Fix mnemonic handling

We were not handling the case right in which we
want to use underlines, but not use markup. Since
we are now using pango_parse_markup for this case,
we need to escape the xml markup.

Fixes: #4041